forked from sveltejs/kit
-
Notifications
You must be signed in to change notification settings - Fork 1
[pull] main from sveltejs:main #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
pull
wants to merge
1,155
commits into
igecloudsdev:main
Choose a base branch
from
sveltejs:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+37,425
−41,625
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR ensures that this server-load data request is also patchable by basically using the same mechanism introduced in #10009
fixes #13318 This PR decodes the URL when the hash router is enabled and the user enters the URL through the browser address bar (during app initialisation and the hashchange event). We need to do this because Safari seems to encode any extra hashes in the URL (only when accessed through the file protocol?). So, entering /#/#about in the address bar becomes /#/%23about which the client router incorrectly thinks is a route instead of just an ID.
closes #10659 --------- Co-authored-by: Tee Ming <chewteeming01@gmail.com> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
entry file names and asset names are untouched. chunk file names are often misleading because they may contain more (or less) than what the filename suggests.
support absolute URLs with `data-sveltekit-preload-code="viewport"` and reroutes for preloadCode in general --------- Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Co-authored-by: Chew Tee Ming <chew.tee.ming@nindatech.com> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
…pt (#13304) since pnpm 10 they don't run at all, and previously they didn't run after the first install either for pnpm users
fixes #6720 This PR changes the inlined stylesheet from client to server so that the paths for imported assets in the CSS such as fonts are correct when the document first loads. In comparison, the client stylesheet links are relative by default, so they always link to the wrong place
Decoding the pathname etc is irrelevant for stripping the internal data that shows us whether or not this is a data request, so move that logic before the reroute hook. fixes #11625 --------- Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* chore: upgrade to vitest 3 * increase timeout * more timeout * more * more * more
…13356) fixes #13320 This PR enables relative hash links to be used with the hash router so that links such as #example get recognised as a link to the current page. We do this by assuming links that have a hash value starting with #/ links to a route while anything else is probably linking to an element with an id on the current page.
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ering (#12052) Previously Netlify Edge Functions didn't support a way to configure path matching that *excludes* paths. Now that it does, we can avoid running the "render" edge function on static assets. Currently, it's [just a no-op](https://github.com/sveltejs/kit/blob/80386437030c5c79913e859e3c32fd194613e1b6/packages/adapter-netlify/src/edge.js#L18-L22), but it's still nice to avoid the invocation. Fixes #5235. --------- Co-authored-by: Simon Holthausen <simon.holthausen@vercel.com> Co-authored-by: Chew Tee Ming <chew.tee.ming@nindatech.com> Co-authored-by: Tee Ming <chewteeming01@gmail.com>
closes #7793 --------- Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Co-authored-by: Tee Ming <chewteeming01@gmail.com>
* docs: handle immutable headers error * Update documentation/docs/30-advanced/20-hooks.md Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com> --------- Co-authored-by: Tee Ming <chewteeming01@gmail.com> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fix link to server hooks locals
…outing (#13884) * changeset, fix, and tests * Update packages/kit/src/runtime/client/client.js Co-authored-by: Elliott Johnson <sejohnson@torchcloudconsulting.com> * Update packages/kit/src/runtime/client/client.js * format * lint * restore original logic --------- Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Co-authored-by: Elliott Johnson <sejohnson@torchcloudconsulting.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* chore: Correct transport docs * Update documentation/docs/30-advanced/20-hooks.md
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
… effect (#13914) * fix: prevent infinite loop when calling `pushState`/`replaceState` in effect * oops * fix duplicate import * test for shallow navigation with effects * remove todo comment --------- Co-authored-by: Chew Tee Ming <chew.tee.ming@nindatech.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* remove types folder * add ts as input
* add vps 6 as peer dep * remove non-existant types directory * Revert "remove non-existant types directory" This reverts commit d286f75.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* fix: use fallback if `untrack` doesn't exist in svelte package * fix
…vite plugin (#13905) * fix: warning for chrome devtools requests now suggests sv instead of vite plugin * Update packages/kit/src/runtime/server/respond.js Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> * Update packages/kit/src/runtime/server/respond.js Co-authored-by: Rich Harris <hello@rich-harris.dev> * Update packages/kit/src/runtime/server/respond.js Co-authored-by: Dominik G. <dominik.goepel@gmx.de> * Update packages/kit/src/runtime/server/respond.js Co-authored-by: Dominik G. <dominik.goepel@gmx.de> --------- Co-authored-by: Ben McCann <322311+benmccann@users.noreply.github.com> Co-authored-by: Rich Harris <hello@rich-harris.dev> Co-authored-by: Dominik G. <dominik.goepel@gmx.de>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* add tests * format * Update packages/kit/test/apps/basics/src/routes/treeshaking/dev/+page.svelte Co-authored-by: Dominik G. <dominik.goepel@gmx.de> * Update packages/kit/test/apps/basics/src/routes/treeshaking/browser/+page.svelte Co-authored-by: Dominik G. <dominik.goepel@gmx.de> * fix * ok this is the one * Update server.test.js --------- Co-authored-by: Dominik G. <dominik.goepel@gmx.de> Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
* Create CHANGELOG-pre-1.md * Update CHANGELOG-pre-1.md * Update CHANGELOG.md
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )